From: Giovanni Campagna <gcampagna@src.gnome.org>
Date: Thu, 14 Feb 2013 16:13:13 +0000 (+0100)
Subject: ScreenShield: don't add the source actor directly to the boxlayout
X-Git-Tag: archive/raspbian/50.5-1+rpi1~1^2~1013^2^2~4813
X-Git-Url: https://dgit.raspbian.org/%22http://www.example.com/cgi/%22/%22http:/www.example.com/cgi/%22?a=commitdiff_plain;h=b45a2d7335e016bc5081d25b6e1a695f558072ff;p=gnome-shell.git

ScreenShield: don't add the source actor directly to the boxlayout

StBoxLayout always fills on the orthogonal direction, so the icon
becomes distorted as the layout grows to accomodate more details.
Instead, use a bin that aligns at the start.

https://bugzilla.gnome.org/show_bug.cgi?id=693822
---

diff --git a/js/ui/screenShield.js b/js/ui/screenShield.js
index 0329381ae..7e693be2c 100644
--- a/js/ui/screenShield.js
+++ b/js/ui/screenShield.js
@@ -204,7 +204,10 @@ const NotificationsBox = new Lang.Class({
 
     _makeNotificationDetailedSource: function(source, box) {
         let sourceActor = new MessageTray.SourceActor(source, SUMMARY_ICON_SIZE);
-        box.add(sourceActor.actor, { y_fill: true });
+        let sourceBin = new St.Bin({ y_align: St.Align.START,
+                                     x_align: St.Align.START,
+                                     child: sourceActor.actor });
+        box.add(sourceBin);
 
         let textBox = new St.BoxLayout({ vertical: true });
         box.add(textBox, { y_fill: false, y_align: St.Align.START });